]> dgit.raspbian.org Git - gnome-shell.git/commitdiff
ci: Set variables at job level where possible
authorFlorian Müllner <fmuellner@gnome.org>
Tue, 10 Jun 2025 11:43:12 +0000 (13:43 +0200)
committerFlorian Müllner <fmuellner@gnome.org>
Tue, 10 Jun 2025 12:52:33 +0000 (14:52 +0200)
Variables that are defined at the global level are more disconnected
from the jobs that use them, and therefore more likely to get outdated
when the job changes.

To litigate that, define variables locally with the job unless they
are used by multiple jobs.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3745>

.gitlab-ci.yml

index b42791f17b72bebb3be457e157da2f628ec26eba..94044cc99c014ca5d7dbeef006fe5c2b626cb19f 100644 (file)
@@ -35,9 +35,6 @@ default:
 
 variables:
   MUTTER_CI_IMAGE: registry.gitlab.gnome.org/gnome/mutter/fedora/42:x86_64-2025-06-05.umockdev
-  FDO_UPSTREAM_REPO: GNOME/gnome-shell
-  BUNDLE: "extensions-git.flatpak"
-  LINT_LOG: "eslint-report.xml"
   TARBALL_ARTIFACT_PATH: build/meson-dist/$CI_PROJECT_NAME-$CI_COMMIT_TAG.tar.xz
 
 .pipeline-guard:
@@ -50,6 +47,7 @@ variables:
 
 .gnome-shell.fedora:
   variables:
+    FDO_UPSTREAM_REPO: GNOME/gnome-shell
     FDO_DISTRIBUTION_VERSION: 42
     FDO_DISTRIBUTION_TAG: '2025-04-01.0'
     FDO_DISTRIBUTION_PACKAGES:
@@ -142,6 +140,8 @@ eslint:
     - .fdo.distribution-image@fedora
     - .gnome-shell.fedora
   stage: review
+  variables:
+    LINT_LOG: "eslint-report.xml"
   script:
     - export NODE_PATH=$(npm root -g)
     - ./.gitlab-ci/run-eslint --output-file ${LINT_LOG} --format junit --stdout
@@ -264,6 +264,7 @@ flatpak:
     RUNTIME_REPO: "https://nightly.gnome.org/gnome-nightly.flatpakrepo"
     FLATPAK_MODULE: "gnome-extensions-app"
     APP_ID: "org.gnome.Extensions.Devel"
+    BUNDLE: "extensions-git.flatpak"
   extends: .flatpak
 
 nightly: